Skip to content

use clean_dir instead of remove_dir on installation directory in generic Tarball easyblock - #3926

Merged
boegel merged 1 commit into
easybuilders:developfrom
smoors:20250915153542_new_pr_tarball
Apr 8, 2026
Merged

use clean_dir instead of remove_dir on installation directory in generic Tarball easyblock#3926
boegel merged 1 commit into
easybuilders:developfrom
smoors:20250915153542_new_pr_tarball

Conversation

@smoors

@smoors smoors commented Sep 15, 2025 •
edited by boegel
Loading

Copy link
Copy Markdown
Contributor

@smoors smoors mentioned this pull request Sep 15, 2025
Closed
@smoors

smoors commented Sep 16, 2025

Copy link
Copy Markdown
Contributor Author

Test report by @smoors

Overview of tested easyconfigs (in order)

  • SUCCESS ecBuild-3.8.5-GCCcore-13.3.0.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node601.hydra.os - Linux Rocky Linux 9.6, x86_64, AMD EPYC 9384X 32-Core Processor (zen4), Python 3.9.21
See https://gist.github.com/smoors/15e55f5bd6634028e808945765f18d5e for a full test report.

@smoors

smoors commented Sep 16, 2025

Copy link
Copy Markdown
Contributor Author

Test report by @smoors

Overview of tested easyconfigs (in order)

  • SUCCESS Go-1.25.0.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node601.hydra.os - Linux Rocky Linux 9.6, x86_64, AMD EPYC 9384X 32-Core Processor (zen4), Python 3.9.21
See https://gist.github.com/smoors/a232f73312b84f1f2508896ddfc8f987 for a full test report.

@smoors

smoors commented Oct 31, 2025

Copy link
Copy Markdown
Contributor Author

@boegelbot please test @ jsc-zen3
EB_ARGS="CESM-deps-2-foss-2023a.eb VSCode-1.104.1-GCCcore-13.3.0.eb samclip-0.4.0-GCCcore-13.3.0.eb pyMBE-0.8.0-foss-2023b.eb MitoHiFi-3.2-foss-2022b.eb MitoHiFi-3.2-foss-2022b.eb --installpath=/tmp/$USER/ebpr-3926"

@boegelbot

boegelbot commented Oct 31, 2025

Copy link
Copy Markdown

@smoors: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=3926 EB_ARGS="CESM-deps-2-foss-2023a.eb VSCode-1.104.1-GCCcore-13.3.0.eb samclip-0.4.0-GCCcore-13.3.0.eb pyMBE-0.8.0-foss-2023b.eb MitoHiFi-3.2-foss-2022b.eb MitoHiFi-3.2-foss-2022b.eb --installpath=/tmp/$USER/ebpr-3926" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3926 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8612

Test results coming soon (I hope)...

Details

- notification for comment with ID 3473547279 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot

boegelbot commented Oct 31, 2025

Copy link
Copy Markdown

Test report by @boegelbot

Overview of tested easyconfigs (in order)

Build succeeded for 6 out of 8 (6 easyconfigs in total)
jsczen3c2.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/ee674ee82ff6b3a135a86d8ab7d6016a for a full test report.

@smoors

smoors commented Oct 31, 2025

Copy link
Copy Markdown
Contributor Author

@boegelbot please test @ jsc-zen3
EB_ARGS="MitoHiFi-3.2-foss-2022b.eb --from-pr 24435 --installpath=/tmp/$USER/ebpr-3926"

@boegelbot

boegelbot commented Oct 31, 2025

Copy link
Copy Markdown

@smoors: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=3926 EB_ARGS="MitoHiFi-3.2-foss-2022b.eb --from-pr 24435 --installpath=/tmp/$USER/ebpr-3926" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3926 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8623

Test results coming soon (I hope)...

Details

- notification for comment with ID 3473810941 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot boegelbot mentioned this pull request Oct 31, 2025
Closed
@smoors

smoors commented Nov 1, 2025

Copy link
Copy Markdown
Contributor Author

@boegelbot please test @ jsc-zen3
EB_ARGS="MitoHiFi-3.2-foss-2022b.eb --installpath=/tmp/$USER/ebpr-3926"

@boegelbot

boegelbot commented Nov 1, 2025

Copy link
Copy Markdown

@smoors: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=3926 EB_ARGS="MitoHiFi-3.2-foss-2022b.eb --installpath=/tmp/$USER/ebpr-3926" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3926 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8624

Test results coming soon (I hope)...

Details

- notification for comment with ID 3476103914 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot

boegelbot commented Nov 1, 2025

Copy link
Copy Markdown

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS CD-HIT-4.8.1-GCC-12.2.0.eb
  • SUCCESS bcbio-gff-0.7.0-foss-2022b.eb
  • SUCCESS MitoHiFi-3.2-foss-2022b.eb

Build succeeded for 3 out of 3 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.6, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/144f9c4e694722ff657ee46b2a4a929d for a full test report.

@smoors smoors mentioned this pull request Feb 20, 2026
Merged
@boegel boegel added this to the next release (5.2.2?) milestone Mar 19, 2026
@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for Tarball easyblock use clean_dir instead of remove_dir on installation directory for Tarball easyblock Mar 19, 2026
@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for Tarball easyblock use clean_dir instead of remove_dir on installation directory for Tarball easyblock Mar 19, 2026
@boegel boegel changed the title use clean_dir instead of remove_dir on installation directory for Tarball easyblock use clean_dir instead of remove_dir on installation directory in generic Tarball easyblock Mar 19, 2026
boegel
boegel approved these changes Apr 8, 2026
View reviewed changes

@boegel boegel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@boegel
boegel merged commit f5d9885 into easybuilders:develop Apr 8, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

5.3.0

Development

Successfully merging this pull request may close these issues.

3 participants

@smoors @boegelbot @boegel